Skip to content

feat(search|a11y): announce results count, optional autofocus#1096

Merged
ThibaudDauce merged 3 commits into
mainfrom
feat/global-search/a11y
May 26, 2026
Merged

feat(search|a11y): announce results count, optional autofocus#1096
ThibaudDauce merged 3 commits into
mainfrom
feat/global-search/a11y

Conversation

@abulte
Copy link
Copy Markdown
Contributor

@abulte abulte commented May 25, 2026

Two features to improve a11y downstream:

  • optional autofocus on search input field (udata-front-kit focuses on a11y quick links)
  • emit an event with results count (for screen readers announcements)

@abulte abulte marked this pull request as ready for review May 25, 2026 14:06
@abulte abulte requested review from ThibaudDauce and nicolaskempf57 and removed request for ThibaudDauce May 25, 2026 14:06
Comment thread datagouv-components/src/components/Search/GlobalSearch.vue Outdated
@abulte abulte changed the base branch from chore/chart-entrypoint to main May 26, 2026 08:12
@abulte abulte force-pushed the feat/global-search/a11y branch from d98d57c to 6b5cde4 Compare May 26, 2026 08:14
@abulte
Copy link
Copy Markdown
Contributor Author

abulte commented May 26, 2026

@ThibaudDauce I don't think we need #1095 for this to work with a production build (I just can't test locally), so I rebased on main and I'm keen on testing the dev version on udata-front-kit when it's merged.

@ThibaudDauce ThibaudDauce merged commit e4ad559 into main May 26, 2026
8 checks passed
@ThibaudDauce ThibaudDauce deleted the feat/global-search/a11y branch May 26, 2026 08:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants